Skip to content

Conversation

khoa-nguyen-18
Copy link
Contributor

  • Update hal_renesas revision to get the FSP 6.2.0 migration commits
  • Update conditions for continuing data transfer for drivers/spi/spi_b_renesas_ra8.c
  • Remove redundant condition check causing transfer error for drivers/i2s/i2s_renesas_ra_ssie.c
  • Add pwm_test alias for Renesas devices

Copy link

github-actions bot commented Oct 18, 2025

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
hal_renesas zephyrproject-rtos/hal_renesas@5ab2c84 zephyrproject-rtos/hal_renesas@f12f062 zephyrproject-rtos/[email protected]

All manifest checks OK

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@github-actions github-actions bot added manifest manifest-hal_renesas DNM (manifest) This PR should not be merged (controlled by action-manifest) labels Oct 18, 2025
@zephyrbot zephyrbot added platform: Renesas RA Renesas Electronics Corporation, RA platform: Renesas Renesas area: Tests Issues related to a particular existing or missing test area: SPI SPI bus area: I2S area: PWM Pulse Width Modulation labels Oct 18, 2025
thenguyenyf
thenguyenyf previously approved these changes Oct 20, 2025
Copy link
Contributor

@thenguyenyf thenguyenyf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Please update the manifest revision

Update rev of hal_renesas to latest

Signed-off-by: Khoa Tran <[email protected]>
When performing polling-based data transfer without enabling interrupts,
the current implementation stops transferring as soon as either the TX or
RX buffer becomes NULL. This causes the transfer to stop prematurely,
even if the other direction still has data to send or receive.

This commit fixes the condition so that data transfer continues
as long as one direction (TX or RX) still has data remaining.

Signed-off-by: Khoa Tran <[email protected]>
When enabling the DTC (Data Transfer Controller) for the I2S SSIE driver
on Renesas RA8x2 boards, a redundant condition check prevented the
interrupt service routines from entering  during data transfer. This
caused the transfer to fail when DTC was active.

This commit removes the unnecessary condition check, allowing the DTC to
operate correctly with I2S SSIE transfers on RA8x2 devices.

Signed-off-by: Khoa Tran <[email protected]>
Add pwm_test alias to support test pwm_api on Renesas RA devices

Signed-off-by: Khoa Tran <[email protected]>
@khoa-nguyen-18 khoa-nguyen-18 force-pushed the renesas_migrate_fsp_620 branch from 215f104 to 660c19e Compare October 21, 2025 02:57
@github-actions github-actions bot removed the DNM (manifest) This PR should not be merged (controlled by action-manifest) label Oct 21, 2025
@zephyrbot zephyrbot requested a review from nashif October 21, 2025 02:58
Copy link

Copy link

@KhiemNguyenT KhiemNguyenT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: I2S area: PWM Pulse Width Modulation area: SPI SPI bus area: Tests Issues related to a particular existing or missing test manifest manifest-hal_renesas platform: Renesas RA Renesas Electronics Corporation, RA platform: Renesas Renesas

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants